Skip to content

Conversation

@qti-mattsinc
Copy link
Contributor

Description

Small change to allow QNN Preprocess to allow a Mul node (with A=B) instead of a Pow node (with Y=2) for layernorm fusion.

@HectorSVC HectorSVC added the ep:QNN issues related to QNN exeution provider label Sep 16, 2025
@HectorSVC
Copy link
Contributor

/azp run Linux QNN CI Pipeline,Win_TRT_Minimal_CUDA_Test_CI,Windows ARM64 QNN CI Pipeline,Windows GPU Doc Gen CI Pipeline

@azure-pipelines
Copy link

Azure Pipelines successfully started running 4 pipeline(s).

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR extends the QNN preprocessor's LayerNorm fusion capability to recognize an alternate pattern where a Mul node (with both inputs being the same tensor) is used instead of a Pow node (with exponent 2.0) for computing the squared values. This is mathematically equivalent since x² = x * x, and some model exporters may generate this pattern.

Key Changes:

  • Added documentation for the Mul-based LayerNorm fusion pattern
  • Extended pattern matching to recognize Mul nodes in place of Pow nodes
  • Added validation logic to ensure Mul nodes have matching inputs

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Member

@yuslepukhin yuslepukhin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@yuslepukhin
Copy link
Member

Please, rebase off main

@qti-mattsinc qti-mattsinc force-pushed the dev/mattsinc/AISW-148002 branch from 71d4556 to 1161fcd Compare January 12, 2026 23:28
@qti-mattsinc qti-mattsinc force-pushed the dev/mattsinc/AISW-148002 branch from 1161fcd to ebaa0cd Compare January 12, 2026 23:29
@qti-mattsinc
Copy link
Contributor Author

Rebased on top of tip

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ep:QNN issues related to QNN exeution provider

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants